Skip to content

Conversation

@Timeline8
Copy link
Contributor

Links are now all dead links.

Links are now all dead links.
Copy link
Contributor

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is an archived version available here: https://web.archive.org/web/20210727135908/http://daveastels.com/trinket-m0-express-hack.html It's missing some of the original images, but it's got a lot of information for anyone who would want to create one of these.

I think it'd be good to link to that one and note it in the text as an archived version.

Changed link to archived version as suggested. Thanks.
@Timeline8
Copy link
Contributor Author

Updated changes with new archived URL. Not sure if I did it right for the Board URL line though.

Copy link
Contributor Author

@Timeline8 Timeline8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated links to use the archived version of the old web page no longer online.

@Timeline8 Timeline8 requested a review from FoamyGuy October 23, 2024 00:08
manufacturer: "Dave Astels"
board_url:
- "http://daveastels.com/trinket-m0-express-hack.html"
board_url: https://web.archive.org/web/20210727135908/http://daveastels.com/trinket-m0-express-hack.html
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's best to make formatting match all of the other device config files with the board URL on it's own line with a hyphen at the beginning.

 - "https://web.archive.org/web/20210727135908/http://daveastels.com/trinket-m0-express-hack.html"

I'm not entirely certain if that syntax is 100% necessary, and I also don't know for sure where this board_url value gets used, but I think keeping it consistent across devices whenever possible is best.

Copy link
Contributor Author

@Timeline8 Timeline8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed. Looks good to me.

Moved URL to next line with indented hyphen.
Didn't look the same as before. 2nd attempt to make it match.
Copy link
Contributor Author

@Timeline8 Timeline8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK I think it looks good now.

Copy link
Contributor

@FoamyGuy FoamyGuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@FoamyGuy FoamyGuy merged commit dff05f0 into adafruit:main Oct 23, 2024
2 checks passed
@Timeline8 Timeline8 deleted the patch-1 branch October 23, 2024 23:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants